Connect PopupMenuManager to destroy signal

PopupMenuManager now connects to the destroy signal of PopupMenu, so
that destroying a PopupMenu is now enough for removing it from the
relevant manager. Useful if menu come and go on-the-fly (like in
app views or sidebars).

https://bugzilla.gnome.org/show_bug.cgi?id=630914

diff --git a/js/ui/popupMenu.js b/js/ui/popupMenu.js
index 89f56e71df81ac3028b80a471aca7725a15bba22..de8de1ecaebb4aacf590b4272de1eed7150399c1 100644 (file)
--- a/js/ui/popupMenu.js
+++ b/js/ui/popupMenu.js
@@ -678,6 +678,7 @@ PopupMenuManager.prototype = {
             menu:              menu,
             openStateChangeId: menu.connect('open-state-changed', Lang.bind(this, this._onMenuOpenState)),
             activateId:        menu.connect('activate', Lang.bind(this, this._onMenuActivated)),
+            destroyId:         menu.connect('destroy', Lang.bind(this, this._onMenuDestroy)),
             enterId:           0,
             buttonPressId:     0
         };